Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

local feature size for point set processing #8006

Open
wants to merge 39 commits into
base: master
Choose a base branch
from
Open

local feature size for point set processing #8006

wants to merge 39 commits into from

Conversation

bizerfr
Copy link
Member

@bizerfr bizerfr commented Jan 29, 2024 •
edited by afabri
Loading

Summary of Changes

A new function estimate_local_feature_size() for the Point_set_processing_3 package. It takes as input a point cloud with or without normals. Then function will return an estimated local feature size for each point.

Release Management

  • Affected package(s): Point_set_processing_3
  • Issue(s) solved (if any): N.A.
  • Feature/Small Feature (if any): A function to estimate local feature size
  • Link to compiled documentation (obligatory for small feature) wrong link name to be changed
  • License and copyright ownership:

@afabri
Copy link
Member

afabri commented Jan 29, 2024

@bizerfr please add the reference manual documentation for the function.

@afabri
Copy link
Member

afabri commented Jan 29, 2024 •
edited
Loading

Also please write a testsuite. Also for the Point_set_3 class.

@afabri
Copy link
Member

afabri commented Jan 29, 2024

As you can see in the continuous integration you have to remove trailing whitespace.

@bizerfr bizerfr requested a review from afabri March 3, 2024 11:53
@bizerfr bizerfr marked this pull request as draft March 4, 2024 18:38
@bizerfr bizerfr marked this pull request as ready for review March 4, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@afabri afabri Awaiting requested review from afabri

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bizerfr @afabri